Skip to content

chore(sec-core): remerge v0.10.0 branch commits - #2552

Merged
edonyzpc merged 5 commits into
mainfrom
release/agent-sec-core/v0.10.0
Aug 14, 2026
Merged

chore(sec-core): remerge v0.10.0 branch commits#2552
edonyzpc merged 5 commits into
mainfrom
release/agent-sec-core/v0.10.0

Conversation

@yangdao479

Copy link
Copy Markdown
Collaborator

Why

What changed

Related issue

User / Agent impact

Risk and compatibility

  • Public CLI, API, configuration, or documented behavior changed
  • Privileged or security-sensitive behavior changed
  • Cross-component contract changed
  • Migration or rollback guidance is needed

Validation

Documentation and rollback

haosanzi and others added 5 commits August 12, 2026 16:15
The cosh prompt-scanner hook passed the user prompt as a `--text`
argv to `agent-sec-cli scan-prompt`, leaking it via
/proc/<pid>/cmdline to other local users and risking E2BIG
fail-open when a large prompt exceeded ARG_MAX (~2MB).

Pipe the prompt via stdin instead, matching codex/hermes/qoder/
qwen. agent-sec-cli scan-prompt already reads stdin when --text is
absent, so no CLI change is needed. The trace-context test is
updated to drop --text from the expected argv and assert the stdin
kwarg; a new test_prompt_passed_via_stdin_not_argv locks it.

Closes #2373

Assisted-by: Qoder:1.22.1
Signed-off-by: Shirong Hao <shirong@linux.alibaba.com>
The openclaw prompt-scan capability passed the user prompt as a
`--text` argv to `agent-sec-cli scan-prompt`, leaking it via
/proc/<pid>/cmdline to other local users and risking E2BIG
fail-open when a large prompt exceeded ARG_MAX (~2MB).

Pipe the prompt via the existing `stdin` option on callAgentSecCli
instead, matching codex/hermes/qoder/qwen. agent-sec-cli
scan-prompt already reads stdin when --text is absent, so no CLI
change is needed. Unit tests are updated to assert `opts.stdin` and
the absence of `--text` in argv.

Closes #2373

Signed-off-by: Shirong Hao <shirong@linux.alibaba.com>
prompt-scan.ts pipes the prompt via stdin (callAgentSecCli opts.stdin)
to avoid /proc/<pid>/cmdline exposure and ARG_MAX limits, mirroring
codex/hermes/qoder/qwen. The e2e pilot wrapper only read stdin when
the argv carried a --stdin flag (the pii-scan pattern), so scan-prompt
calls with no --text argv left input undefined, skipped the deny
override, and forwarded empty stdin to the real CLI — the plugin
fail-opened and gateway-probes could not find the expected pass/deny
signals.

Read stdin once for scan-prompt when no --text argv is present, use
it for both override matching and forwarding to the real CLI. Backward
compat: scan-code (--code argv) and scan-pii (--stdin flag) unchanged.

Closes #2373

Assisted-by: Qoder:1.22.1
Signed-off-by: Shirong Hao <shirong@linux.alibaba.com>
Bump the component manifest, CLI package, and all six agent plugin manifests to 0.10.1.

Record the two release fixes in CHANGELOG.md and add the matching RPM spec changelog entry.

Version strings and release notes only; no scanner, hook, or daemon behavior changes here.

Signed-off-by: yizheng <YiZheng.Yang@linux.alibaba.com>
@github-actions github-actions Bot added component:sec-core src/agent-sec-core/ scope:documentation ./docs/|./*.md|./NOTICE labels Aug 14, 2026

@qoderai qoderai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

本次审查范围内未发现需要修改的问题。


🤖 Generated by QoderView workflow run

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6000d0058e

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/agent-sec-core/CHANGELOG.md
@edonyzpc
edonyzpc merged commit c122a0a into main Aug 14, 2026
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:sec-core src/agent-sec-core/ scope:documentation ./docs/|./*.md|./NOTICE

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants